Skip to content

Add vendor and image class UUID support #23609

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add vendor and image class UUID support #23609

wants to merge 3 commits into from
+237 −2

Conversation

tomchy
Copy link
Contributor

@tomchy tomchy commented Jul 31, 2025

No description provided.

@tomchy tomchy requested review from a team as code owners July 31, 2025 14:43
@github-actions github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Jul 31, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jul 31, 2025 •
edited by github-actions bot
Loading

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
mcuboot nrfconnect/sdk-mcuboot@e1f2ab3 (main) nrfconnect/sdk-mcuboot#486 nrfconnect/sdk-mcuboot#486/files
zephyr nrfconnect/sdk-zephyr@d4d69a4 (main) nrfconnect/sdk-zephyr#3122 nrfconnect/sdk-zephyr#3122/files

⛔ DNM label due to: 2 projects with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jul 31, 2025 •
edited
Loading

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 5

Inputs:

Sources:

sdk-nrf: PR head: 6e0f4f452ab45116a9277fcfa2db7d1e894e139c
mcuboot: PR head: eb8c3c85ecd8305cb5513802013887ab33d62437
zephyr: PR head: d0ea0546efa0989c38767932326cd1f359bc25f0

more details

sdk-nrf:

PR head: 6e0f4f452ab45116a9277fcfa2db7d1e894e139c
merge base: 96a895a777e891e79931380e7898241770ffcc0e
target head (main): e463ba7ffd2d92ebe8d66cc53c41036e2c66595d
Diff

mcuboot:

PR head: eb8c3c85ecd8305cb5513802013887ab33d62437
merge base: e1f2ab3806ce7ebc7ef34b3fc04272e747590745
target head (main): e1f2ab3806ce7ebc7ef34b3fc04272e747590745
Diff

zephyr:

PR head: d0ea0546efa0989c38767932326cd1f359bc25f0
merge base: d4d69a4a781859dd8691633988b6b8816b653dc9
target head (main): d4d69a4a781859dd8691633988b6b8816b653dc9
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (21) 
bootloader
│  ├── mcuboot
│  │  ├── boot
│  │  │  ├── bootutil
│  │  │  │  ├── include
│  │  │  │  │  ├── bootutil
│  │  │  │  │  │  ├── image.h
│  │  │  │  │  │  │ mcuboot_uuid.h
│  │  │  │  ├── src
│  │  │  │  │  │ image_validate.c
│  │  │  ├── zephyr
│  │  │  │  ├── Kconfig
│  │  │  │  ├── include
│  │  │  │  │  ├── mcuboot_config
│  │  │  │  │  │  │ mcuboot_config.h
│  │  │  │  │ main.c
│  │  ├── docs
│  │  │  │ design.md
│  │  ├── scripts
│  │  │  ├── imgtool
│  │  │  │  ├── image.py
│  │  │  │  │ main.py
samples
│  ├── zephyr
│  │  ├── subsys
│  │  │  ├── mgmt
│  │  │  │  ├── mcumgr
│  │  │  │  │  ├── smp_svr
│  │  │  │  │  │  ├── prj.conf
│  │  │  │  │  │  ├── sysbuild
│  │  │  │  │  │  │  ├── ipc_radio.conf
│  │  │  │  │  │  │  │ mcuboot.conf
subsys
│  ├── CMakeLists.txt
│  ├── bootloader
│  │  ├── Kconfig
│  │  ├── bl_uuid
│  │  │  ├── CMakeLists.txt
│  │  │  ├── Kconfig
│  │  │  ├── Kconfig.uuid.template
│  │  │  │ bl_uuid.c
west.yml
zephyr
│  ├── cmake
│  │  │ mcuboot.cmake
│  ├── modules
│  │  │ Kconfig.mcuboot

Outputs:

Toolchain

Version: c0e69fc6ed
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:c0e69fc6ed_bba2ea5f2e

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 16
    • sdk-zephyr test count: 710
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-chip
    • ✅ test-fw-nrfconnect-tfm
    • ✅ test-sdk-find-my
    • ✅ test-sdk-mcuboot
    • ✅ test-sdk-dfu
    • ⚠️ test-fw-nrfconnect-nrf-iot_cloud
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-thread-main
    • test-low-level
    • test-sdk-audio
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@github-actions GitHub Actions
Copy link

You can find the documentation preview for this PR here.

@tomchy tomchy force-pushed the feature/bootutil/NCSDK-34175_Add_vid_cid_support branch 2 times, most recently from 4ab54b4 to c98c5f6 Compare August 1, 2025 16:03
@tomchy tomchy force-pushed the feature/bootutil/NCSDK-34175_Add_vid_cid_support branch from c98c5f6 to 943c8e9 Compare August 29, 2025 15:54
@tomchy
manifest: Update sdk-zephyr and sdk-mcuboot
beeb6b3 
Include the VID and CID feature.

Ref: NCSDK-34175

Signed-off-by: Tomasz Chyrowicz <[email protected]>
@tomchy
bootloader: Add basic UUID checks implementation
8782a62 
Provide an implementation for MCUboot UUID checks that specify a single,
common vendor identifier and a unique class identifier for each image.

Ref: NCSDK-34175

Signed-off-by: Tomasz Chyrowicz <[email protected]>
@tomchy
sample: Enable VID and CID checks
6e0f4f4 
Enable VID and CID checks inside smp_svr sample for nRF54H20.

Ref: NCSDK-34175

Signed-off-by: Tomasz Chyrowicz <[email protected]>
@tomchy tomchy force-pushed the feature/bootutil/NCSDK-34175_Add_vid_cid_support branch from 943c8e9 to 6e0f4f4 Compare September 2, 2025 13:22
@tomchy tomchy requested review from a team as code owners September 2, 2025 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ahasztag ahasztag ahasztag approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. DNM manifest manifest-mcuboot manifest-zephyr
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tomchy @NordicBuilder @ahasztag